Skip to content

fix(header): add genui-sdk of docs and playground#84

Merged
kagol merged 5 commits intodevfrom
fix-header-data
Feb 28, 2026
Merged

fix(header): add genui-sdk of docs and playground#84
kagol merged 5 commits intodevfrom
fix-header-data

Conversation

@James-9696
Copy link
Collaborator

@James-9696 James-9696 commented Feb 10, 2026 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

  • New Features
    • Added GenUI-SDK menu items in the Playground and Documentation sections, enabling quick access to generative UI resources and related guides.

@coderabbitai
Copy link

coderabbitai bot commented Feb 10, 2026 •
edited
Loading

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 866be50 and 9d65a74.

📒 Files selected for processing (1)
  • packages/common/src/config/header.ts
🚧 Files skipped from review as they are similar to previous changes (1)
  • packages/common/src/config/header.ts

Walkthrough

Adds two GenUI-SDK menu entries to the header configuration file. One entry appears in the "演练场" (Playground) section and another in the "开发文档" (Development Documentation) section, each with corresponding links and metadata.

Changes

Cohort / File(s) Summary
Configuration Menu Items
packages/common/src/config/header.ts		 Added two GenUI-SDK entries to nested menu structures: one under the Playground section linking to the design playground, and one under Development Documentation linking to the quick-start guide. Both include title, description, logo, and GitHub repository links.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 In header lands, where menus grow,
Two GenUI paths now brightly show,
A playground hub and docs so true,
Fresh menu items, hopping through! 🌟

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding GenUI-SDK entries to both the docs and playground sections of the header configuration.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-header-data

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@kagol
Copy link
Member

kagol commented Feb 10, 2026 •
edited
Loading

@James-9696 顺序上建议放 NEXT-SDK 后面，保持一致。
另外可以把 NEXT-SDK 的 playground 也加上哈，链接：https://playground.opentiny.design/next-sdk

@James-9696
Copy link
Collaborator Author

James-9696 commented Feb 10, 2026

@James-9696 顺序上建议放 NEXT-SDK 后面，保持一致。 另外可以把 NEXT-SDK 的 playground 也加上哈，链接：https://playground.opentiny.design/next-sdk

已修改

coderabbitai[bot]
coderabbitai bot reviewed Feb 10, 2026
View reviewed changes
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
packages/common/src/config/header.ts (1)

108-122: ⚠️ Potential issue | 🟡 Minor

Duplicate NEXT-SDKs entry in the Playground section.

Lines 108–115 already define a NEXT-SDKs playground item (with hide: true), and lines 116–122 add an identical one without hide. This will render two NEXT-SDKs entries — one hidden and one visible. You likely intended to replace the old entry rather than add a second one.

Proposed fix: remove the old hidden entry 
       {
         title: 'TinyRobot',
         desc: '智能助手',
         href: 'https://playground.opentiny.design/tiny-robot',
         logo: tinyRobot,
         github: 'https://github.com/opentiny/playground'
       },
-      {
-        title: 'NEXT-SDKs',
-        desc: '开发套件',
-        href: 'https://playground.opentiny.design/next-sdk',
-        hide: true,
-        logo: tinySdks,
-        github: 'https://github.com/opentiny/playground'
-      },
       {
         title: 'NEXT-SDKs',
         desc: '开发套件',
         href: 'https://playground.opentiny.design/next-sdk',
         logo: tinySdks,
         github: 'https://github.com/opentiny/playground'
       },

kagol
kagol reviewed Feb 28, 2026
View reviewed changes
kagol
kagol previously approved these changes Feb 28, 2026
View reviewed changes
@kagol kagol merged commit e298da6 into dev Feb 28, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

@kagol kagol kagol approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@James-9696 @kagol